Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow fakers to build on previously populated fields. #420

Merged
merged 14 commits into from
Aug 16, 2021

Conversation

prescod
Copy link
Contributor

@prescod prescod commented Jun 20, 2021

Allow fakers to build on previously populated fields, such as emails based on firstnames.

Currently: Emails and Usernames are based on FirstNames and LastNames.

Also adds a bit of test coverage.

@prescod prescod requested a review from a team as a code owner June 20, 2021 16:14
@prescod prescod force-pushed the feature/contextual-fakers branch from 6250df2 to 58d3411 Compare July 27, 2021 08:24
Copy link

@boakley boakley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have a video call where we went over a few small changes. Let me know when you've done them and I'll make a final pass. Otherwise, this looks good to me.

snowfakery/fakedata/fake_data_generator.py Outdated Show resolved Hide resolved
snowfakery/fakedata/fake_data_generator.py Show resolved Hide resolved
snowfakery/fakedata/fake_data_generator.py Outdated Show resolved Hide resolved
@prescod prescod requested a review from boakley August 6, 2021 13:30
Copy link

@boakley boakley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After having gone over the code with Paul, and seeing these last few changes, I'm satisfied.

Copy link
Contributor

@davisagli davisagli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are any updates to documentation needed?

@prescod
Copy link
Contributor Author

prescod commented Aug 16, 2021

@davisagli : I think I'd just consider this "making fake data more realistic" and not needing of docs unless it causes some confusion or problem for someone.

I'll also wait on documenting the new plugin local variables feature until someone asks for a "feature like local variables for plugins". Premature documentation locks me into a particular interface.

@prescod prescod merged commit 02a9cec into main Aug 16, 2021
@prescod prescod deleted the feature/contextual-fakers branch August 16, 2021 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants